Skip to content

Do not use remote-provider-cert ops file#597

Merged
ystros merged 1 commit into
ubuntu-jammyfrom
drop-remove-provider-cert
May 8, 2026
Merged

Do not use remote-provider-cert ops file#597
ystros merged 1 commit into
ubuntu-jammyfrom
drop-remove-provider-cert

Conversation

@ystros

@ystros ystros commented May 8, 2026

Copy link
Copy Markdown
Contributor

Change is already flown

@ystros @aramprice
Do not use remote-provider-cert ops file
98d1570 
Signed-off-by: Aram Price <aram.price@broadcom.com>
@coderabbitai

coderabbitai Bot commented May 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 57ca223e-0a73-4be9-bbec-bd09db4925b4

📥 Commits

Reviewing files that changed from the base of the PR and between 6596133 and 98d1570.

📒 Files selected for processing (1)
  • ci/pipelines/publisher.yml
💤 Files with no reviewable changes (1)
  • ci/pipelines/publisher.yml

Walkthrough

This PR updates the CI pipeline configuration for the stemcell upload test workflow. The run_stemcell_upload_tests job's prepare-director task modifies its list of optional ops files by removing remove-provider-cert.yml and adding remove-hm.yml, external-ip-with-registry-not-recommended.yml, and misc/source-releases/bosh.yml instead. This changes which operational transformations are applied during the director preparation phase of stemcell testing.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is vague and incomplete. It does not follow the required merge-forward strategy template or provide meaningful context about the change. Provide a more detailed description explaining why the remote-provider-cert ops file is being removed and ensure the PR follows the merge-forward strategy outlined in the template.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: removing the remote-provider-cert ops file from the workflow configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch drop-remove-provider-cert

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ystros ystros merged commit 4a792ed into ubuntu-jammy May 8, 2026
12 checks passed
@ystros ystros deleted the drop-remove-provider-cert branch May 8, 2026 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Alphasite Alphasite Alphasite approved these changes

@KauzClay KauzClay KauzClay approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ystros @Alphasite @KauzClay